Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding more Add data logos to Icons #341

Merged
merged 7 commits into from
Jan 29, 2018
Merged

Adding more Add data logos to Icons #341

merged 7 commits into from
Jan 29, 2018

Conversation

formgeist
Copy link
Contributor

Closes elastic/kibana#16279

screenshot 2018-01-29 10 17 10

Adding Docker, Kubernetes and Redis logos to the Icons/Logos list for the Add data UI in Kibana.

@formgeist formgeist self-assigned this Jan 29, 2018
@formgeist formgeist requested a review from snide January 29, 2018 09:25
@@ -112,7 +112,6 @@ exports[`EuiCodeEditor is rendered 1`] = `
wrap="off"
/>
<div
aria-hidden="true"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@snide Not sure why these are removed, it happens after I run tests.

@@ -17,6 +17,7 @@ exports[`EuiFormRow behavior onBlur is called in child 1`] = `
<EuiFormLabel
htmlFor="generated-id"
isFocused={false}
isInvalid={undefined}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@snide Same goes for these, added after tests.

@snide
Copy link
Contributor

snide commented Jan 29, 2018

You're going to want to clear your cache.

npm run test-unit -- --clearCache

Then rerun the tests.

@formgeist
Copy link
Contributor Author

@snide Right, will do

@formgeist
Copy link
Contributor Author

@snide Odd. The tests pass, but it seems it failed anyways 😟

@snide
Copy link
Contributor

snide commented Jan 29, 2018

Merging. Travis acting up again.

@snide snide merged commit 5052fad into elastic:master Jan 29, 2018
@formgeist formgeist deleted the more-logos branch July 1, 2019 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants